Skip to content

[close #370] make gRPC and TiKV health check configurable (#369)#387

Closed
ti-srebot wants to merge 1 commit intotikv:release-3.1from
ti-srebot:release-3.1-2eb77d689197
Closed

[close #370] make gRPC and TiKV health check configurable (#369)#387
ti-srebot wants to merge 1 commit intotikv:release-3.1from
ti-srebot:release-3.1-2eb77d689197

Conversation

@ti-srebot
Copy link
Copy Markdown
Collaborator

@ti-srebot ti-srebot commented Dec 10, 2021 •
edited
Loading

cherry-pick #369 to release-3.1
You can switch your code base to this Pull Request by using git-extras:

# In client-java repo:
git pr https://github.com/tikv/client-java/pull/387

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/client-java.git pr/387:release-3.1-2eb77d689197

Fix warning in OLAP engine:

2021-12-08 05:18:47,334 WARN  org.tikv.common.region.StoreHealthyChecker                   [] - store [tikv_host:tikv_port] is not reachable
2021-12-08 05:19:01,130 WARN  org.tikv.common.region.AbstractRegionStoreClient             [] - forward request to store [tikv_host:tikv_port] for region[174126738]
2021-12-08 05:19:01,894 WARN  org.tikv.common.region.StoreHealthyChecker                   [] - store [tikv_host:tikv_port] is not reachable
2021-12-08 05:19:01,933 WARN  org.tikv.common.region.StoreHealthyChecker                   [] - store [tikv_host:tikv_port] recovers to be reachable and canforward
2021-12-08 05:19:05,490 WARN  org.tikv.common.region.StoreHealthyChecker                   [] - store [tikv_host:tikv_port] is not reachable
2021-12-08 05:19:05,604 WARN  org.tikv.common.region.StoreHealthyChecker                   [] - store [tikv_host:tikv_port] recovers to be reachable and canforward

@humengyu2012 @ti-srebot
cherry pick tikv#369 to release-3.1
9d50751 
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Copy Markdown
Collaborator Author

ti-srebot commented Dec 10, 2021

/run-all-tests

@ti-srebot ti-srebot mentioned this pull request Dec 10, 2021
Merged
@marsishandsome
Copy link
Copy Markdown
Collaborator

marsishandsome commented Dec 20, 2021

This is not necessary to cherry pick to 3.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sunxiaoguang sunxiaoguang Awaiting requested review from sunxiaoguang

@zz-jason zz-jason Awaiting requested review from zz-jason

Assignees

No one assigned

Labels

component/common

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ti-srebot @marsishandsome @humengyu2012